View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0001831T99X171.00 SKB EagleSW Issuepublic2023-11-10 16:492023-11-17 15:22
Reporter(ALTech) JunGyu Kim Assigned To(SW) Jason LingDue Date2023-11-13 17:37
PriorityhighSeveritys4-minorReproducibilityhave not tried
Status closedResolutionfixed 
Summary0001831: [Smart3][ATV10][VCS] RCU long press operation is slower than INTEK model
DescriptionHi Kerwin,

SKB QA found an issue that the long press operation in RCU is slower than INTEK model(Smart3).
As shown in the attached log, the key time in SKB IME is slower than in INTEK.
Jira ticket : https://jira.skbroadband.com/browse/BPM-25073

INTEK <BIP-AI100>
2023-11-10 10:50:21.045 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:50:21.100 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:50:21.145 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:50:21.196 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:50:21.251 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:50:21.299 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:50:21.347 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:50:21.397 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:50:21.446 1067-1067 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 22, KeyIndex = 15, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false

<BFX-AT100>
2023-11-10 10:48:51.480 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:48:51.585 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:48:51.683 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:48:51.782 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:48:51.888 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:48:51.988 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:48:52.090 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:48:52.188 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false
2023-11-10 10:48:52.288 5073-5073 BTV_IMEService com.skb.btv.ime I ime:onKeyDown keyCode = 21, KeyIndex = 14, KeySets().length = 10, isWebInputType = false, isEditable = false, isTypeNull = true, currentKeyMode = 3, isSearchMode = false, isInputViewShown = false, isAllowApp = false

It seems that there is something like a delay value that transmits the key, so please check why there is a difference.
I have attached log files and video. Please refer it.

Thank you.
Kim
TagsNo tags attached.
Attach Tags

Users monitoring this issue

User List (ALTech) SY Yoon , (ALTech) Younkwang Jung , (SW) Jason Ling

Activities

(ALTech) JunGyu Kim

2023-11-10 16:49

developer  

BID-AT200_20231110-135103.zip (704,599 bytes)
BFX-AT100_20231110-135116.zip (510,297 bytes)
Left_BFX-AT100__Right_BID-AT200.mp4 (36,062,845 bytes)   

(ALTech) Younkwang Jung

2023-11-10 17:39

developer   ~0014507

Hi all

when I check the log , it's seems that RCU repeat key delay is 50ms at INTEK and 100ms at FXN
Could you check the repeat key processing ?

Thank you
YK.Jung

(ALTech) Younkwang Jung

2023-11-13 08:28

developer   ~0014508

Hi all

This issue needs to be confirmed by today. ( it's major issue)
Please check it quickly

Thank you
YK.Jung

(SW) Jason Ling

2023-11-13 10:20

developer   ~0014512

Hi Jungyu,

Please check the following commit, I guess that Intek doesn't have this TVSTORM's commit.

commit abb65950434edd072cc6e17749ede8473390e04a
Author: Jason Ling <jason.tf.ling@foxconn.com>
Date: Sat May 25 11:05:38 2019 +0800

    Change RCU Repeat interval (From TVSTORM)

diff --git a/services/inputflinger/InputDispatcher.cpp b/services/inputflinger/InputDispatcher.cpp
index b921d954d..ecbf39b34 100644
--- a/services/inputflinger/InputDispatcher.cpp
+++ b/services/inputflinger/InputDispatcher.cpp
@@ -727,6 +727,9 @@ void InputDispatcher::resetKeyRepeatLocked() {

 InputDispatcher::KeyEntry* InputDispatcher::synthesizeKeyRepeatLocked(nsecs_t currentTime) {
     KeyEntry* entry = mKeyRepeatState.lastKeyEntry;
+ //TVSTORM ++
+ nsecs_t skbkeyRepeatTimeout = 100 * 1000000LL;
+ //TVSTORM --

     // Reuse the repeated key entry if it is otherwise unreferenced.
     uint32_t policyFlags = entry->policyFlags &
@@ -753,7 +756,10 @@ InputDispatcher::KeyEntry* InputDispatcher::synthesizeKeyRepeatLocked(nsecs_t cu
     // mKeyRepeatState.lastKeyEntry in addition to the one we return.
     entry->refCount += 1;

- mKeyRepeatState.nextRepeatTime = currentTime + mConfig.keyRepeatDelay;
+ //TVSTORM ++
+ //ori. mKeyRepeatState.nextRepeatTime = currentTime + mConfig.keyRepeatDelay;
+ mKeyRepeatState.nextRepeatTime =currentTime + skbkeyRepeatTimeout;
+ //TVSTORM --
     return entry;
 }

Thanks,
Jason

(ALTech) JunGyu Kim

2023-11-13 16:43

developer   ~0014518

Hi Jason,

Our QA tested after revert this commit. It seems no issue.
And SKB wants apply this revert. So, please apply it to main branch.
Please use "BPM-25073" jira number.

Thank you.
Kim

(SW) Jason Ling

2023-11-13 17:23

developer   ~0014520

Hi JunGyu,

Done

2023-11-13 17:20:09 | jason.tf.ling@fii-.. | f8a4bfadf | frameworks/native | [BPM-25073] Revert "Change RCU Repeat interval (From TVSTORM)"

Thanks,
Jason

Issue History

Date Modified Username Field Change
2023-11-10 16:49 (ALTech) JunGyu Kim New Issue
2023-11-10 16:49 (ALTech) JunGyu Kim Status new => assigned
2023-11-10 16:49 (ALTech) JunGyu Kim Assigned To => (SW) Kerwin Chen
2023-11-10 16:49 (ALTech) JunGyu Kim File Added: BID-AT200_20231110-135103.zip
2023-11-10 16:49 (ALTech) JunGyu Kim File Added: BFX-AT100_20231110-135116.zip
2023-11-10 16:49 (ALTech) JunGyu Kim File Added: Left_BFX-AT100__Right_BID-AT200.mp4
2023-11-10 16:50 (ALTech) JunGyu Kim Issue Monitored: (ALTech) SY Yoon
2023-11-10 16:50 (ALTech) JunGyu Kim Issue Monitored: (ALTech) Younkwang Jung
2023-11-10 17:39 (ALTech) Younkwang Jung Note Added: 0014507
2023-11-10 17:40 (ALTech) Younkwang Jung Issue Monitored: (SW) Jason Ling
2023-11-13 08:28 (ALTech) Younkwang Jung Note Added: 0014508
2023-11-13 08:29 (ALTech) Younkwang Jung Assigned To (SW) Kerwin Chen => (ALTech) Younkwang Jung
2023-11-13 08:29 (ALTech) Younkwang Jung Assigned To (ALTech) Younkwang Jung => (SW) Jason Ling
2023-11-13 10:20 (SW) Jason Ling Note Added: 0014512
2023-11-13 16:43 (ALTech) JunGyu Kim Note Added: 0014518
2023-11-13 17:23 (SW) Jason Ling Status assigned => resolved
2023-11-13 17:23 (SW) Jason Ling Resolution open => fixed
2023-11-13 17:23 (SW) Jason Ling Note Added: 0014520
2023-11-17 15:22 (ALTech) JunGyu Kim Status resolved => closed